Skip to content

fix: AI Services provisioning order for private endpoint CognitiveServices/accounts fail#1052

Closed
Ashwal-Microsoft wants to merge 1 commit into
devfrom
Ash-Bicep-Dependson
Closed

fix: AI Services provisioning order for private endpoint CognitiveServices/accounts fail#1052
Ashwal-Microsoft wants to merge 1 commit into
devfrom
Ash-Bicep-Dependson

Conversation

@Ashwal-Microsoft

@Ashwal-Microsoft Ashwal-Microsoft commented Jun 19, 2026

Copy link
Copy Markdown

This pull request refactors how AI service deployments and role assignments are handled in the infra/main.bicep file. The main change is moving the deployment and role assignment configurations from the main AI service module into a dedicated submodule, which improves modularity and maintainability.

Refactoring of AI service deployment and role assignments:

  • Removed inline deployments and roleAssignments properties from the main aiFoundryAiServices module definition, simplifying its configuration.
  • Introduced a new conditional submodule, aiFoundryAiServicesDeployments (using modules/ai-services-deployments.bicep), to handle AI model deployments and role assignments separately. This submodule is only deployed if a new AI Foundry AI Project is being created.
  • Passed the model deployment configuration and required role assignments to the new submodule, preserving the original deployment logic but encapsulating it for better separation of concerns.

Dependency management:

  • Updated dependsOn logic to ensure the correct deployment order when private networking is enabled, and to make the new deployments module depend on the main AI service resource.## Purpose
  • ...

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants